inspector: Do not accidentally change tab in the object notebook
authorStefano Facchini <stefano.facchini@gmail.com>
Sat, 12 Jul 2014 12:59:59 +0000 (14:59 +0200)
committerMatthias Clasen <mclasen@redhat.com>
Sun, 13 Jul 2014 19:19:27 +0000 (15:19 -0400)
The "hide-and-show" dance makes the notebook select the next tab.

https://bugzilla.gnome.org/show_bug.cgi?id=733092

gtk/inspector/classes-list.c
gtk/inspector/size-groups.c
gtk/inspector/style-prop-list.c

index 8bc17d9ebf69f9d96eb45b65ee54e04b168a4fd7..2d21265aaea0d1b68f99c56b0ab8adf3b03708ed 100644 (file)
@@ -235,7 +235,6 @@ cleanup_context (GtkInspectorClassesList *cl)
 
   gtk_list_store_clear (cl->priv->model);
   cl->priv->context = NULL;
-  gtk_widget_hide (GTK_WIDGET (cl));
 }
 
 static void
@@ -245,6 +244,7 @@ remove_dead_object (gpointer data, GObject *dead_object)
 
   cl->priv->context = NULL;
   cleanup_context (cl);
+  gtk_widget_hide (GTK_WIDGET (cl));
 }
 
 void
@@ -258,7 +258,10 @@ gtk_inspector_classes_list_set_object (GtkInspectorClassesList *cl,
   cleanup_context (cl);
 
   if (!GTK_IS_WIDGET (object))
-    return;
+    {
+      gtk_widget_hide (GTK_WIDGET (cl));
+      return;
+    }
 
   gtk_widget_show (GTK_WIDGET (cl));
 
index b6172f80c91cba25d3d401a70c310f3889078f83..8cec9378353649bda7a7b0667353ddd237f7a506 100644 (file)
@@ -271,19 +271,21 @@ gtk_inspector_size_groups_set_object (GtkInspectorSizeGroups *sl,
                                       GObject                *object)
 {
   clear_view (sl);
-  gtk_widget_hide (GTK_WIDGET (sl));
 
-  if (GTK_IS_WIDGET (object))
+  if (!GTK_IS_WIDGET (object))
     {
-      GSList *groups, *l;
-
-      gtk_widget_show (GTK_WIDGET (sl));
-      groups = _gtk_widget_get_sizegroups (GTK_WIDGET (object));
-      for (l = groups; l; l = l->next)
-        {
-          GtkSizeGroup *group = l->data;
-          add_size_group (sl, group);
-        }
+      gtk_widget_hide (GTK_WIDGET (sl));
+      return;
+    }
+
+  GSList *groups, *l;
+
+  gtk_widget_show (GTK_WIDGET (sl));
+  groups = _gtk_widget_get_sizegroups (GTK_WIDGET (object));
+  for (l = groups; l; l = l->next)
+    {
+      GtkSizeGroup *group = l->data;
+      add_size_group (sl, group);
     }
 }
 
index 9fed77b64d10be536d445be7394b1e0eca936308..4212e65bc7b38cbc052c58056c243bfb60cbe1d2 100644 (file)
@@ -308,10 +308,11 @@ void
 gtk_inspector_style_prop_list_set_object (GtkInspectorStylePropList *self,
                                           GObject                   *object)
 {
-  gtk_widget_hide (GTK_WIDGET (self));
-
   if (self->priv->widget == (GtkWidget *)object)
-    return;
+    {
+      gtk_widget_hide (GTK_WIDGET (self));
+      return;
+    }
 
  if (self->priv->widget)
     {
@@ -321,7 +322,10 @@ gtk_inspector_style_prop_list_set_object (GtkInspectorStylePropList *self,
     }
 
   if (!GTK_IS_WIDGET (object))
-    return;
+    {
+      gtk_widget_hide (GTK_WIDGET (self));
+      return;
+    }
 
   self->priv->widget = (GtkWidget *)object;
   g_object_weak_ref (G_OBJECT (self), disconnect_each_other, object);